Skip to content

feat: Add Mixpanel tracking events for audit logs export flow (M2-10770) - #2225

Merged
divbzero merged 30 commits into
developfrom
feat/audit-logs-mixpanel-events-10770
Jul 9, 2026
Merged

feat: Add Mixpanel tracking events for audit logs export flow (M2-10770)#2225
divbzero merged 30 commits into
developfrom
feat/audit-logs-mixpanel-events-10770

Conversation

@adeiji

@adeiji adeiji commented May 29, 2026

Copy link
Copy Markdown
Contributor
  • Tests for the changes have been added
  • Related documentation has been added / updated

📝 Description

🔗 Jira Ticket M2-10770

Adds Mixpanel tracking events across the full audit logs export flow.

Changes include:

  • Add ExportAuditLogsDownload event, tracked when the user clicks the Export button in the audit logs settings popup
  • Add ExportAuditLogsSuccessful event, tracked when the audit logs CSV download completes
  • Add ExportAuditLogsFailed event, tracked when the export encounters an error
  • Add test coverage for all audit log Mixpanel events

🪤 Peer Testing

  • Open an applet and click the Export button in the header. Select Audit Logs.

    Expected outcome: Export Audit Logs click fires in Mixpanel.

  • Select a date range and click the Export button in the modal.

    Expected outcome: Export Audit Logs download fires in Mixpanel.

  • Allow the export to complete successfully.

    Expected outcome: Export Audit Logs Successful fires in Mixpanel

  • Simulate a failure (e.g. disconnect network mid-export) and observe the error modal.

    Expected outcome: Export Audit Logs Failed fires in Mixpanel

adeiji added 25 commits May 5, 2026 21:58
Moved date range functionality into a reusable component
…ePicker component

Remove ExportDateType in favor of DateRangePickerType and replaced inline date range logic with the reusable DateRangePicker component.
Duplicate date-related tests removed as they are now covered by DateRangePicker.test.tsx.
…ecryptedDataSucceed and moved each function props into shared props
… CSV export

- Fix date format sent to API to be date-only instead of datetime
- Fix API response type to match flat array structure
- Update AuditEvent type to match actual API dot-notation keys
- Refactor export to per-page CSV download to avoid memory accumulation
- Add isExportRef guard to prevent duplicate exports
- Add French translations
- Added test
removed tsv boolean form value
fill value now set as a variable for audit logs and response data svgs
added | null for non-required values in auditEvent.ts to match backend

@divbzero divbzero left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@adeiji This one looks good to me! Ready to merge.

@divbzero

Copy link
Copy Markdown
Contributor

@adeiji Could you please update the target branch to dev now that feat/audit-logs has been merged with #2222?

@adeiji
adeiji changed the base branch from feat/audit-logs to develop June 12, 2026 20:39
@divbzero

divbzero commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

As discussed with Ade and Natalia, we will merge this to develop and test there because the Amplify build is not triggering in this CI.

@divbzero divbzero changed the title feat: Add Mixpanel tracking events for audit logs export flow feat: Add Mixpanel tracking events for audit logs export flow (M2-10770) Jul 9, 2026
@divbzero
divbzero merged commit 827e958 into develop Jul 9, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants